Per-row Select button + preview horizontal-overflow fix#51
Merged
eliothedeman merged 2 commits intomainfrom Apr 23, 2026
Merged
Per-row Select button + preview horizontal-overflow fix#51eliothedeman merged 2 commits intomainfrom
eliothedeman merged 2 commits intomainfrom
Conversation
The multi-select UX forced users to scroll to a footer Preview button to
advance; merging traces isn't well supported yet, so the checkboxes had
no real payoff. Each Claude/Pi session row now has a per-row "Select →"
button that derives that one session and jumps straight to Preview.
Drops the now-unused `selected` state, `ClaudeToggleSession`/
`PiToggleSession` messages, and the footer bar. `ClaudeDerive`/`PiDerive`
now carry `{ path, sid }` inline.
…l scroll `.preview-body--split` used `grid-template-columns: 380px 1fr`. Grid 1fr defaults to `minmax(auto, 1fr)`, so an unbreakable string in the transcript (a long file path, a wide pre block) would grow the right track past the viewport and drag the whole preview page into horizontal scroll. Switch to `minmax(0, 1fr)` so the column is capped at the available width and `.chat-view`'s `overflow-x: hidden` can actually clip overflowing bubbles.
|
🔍 Preview deployed: https://fe534ab7.toolpath.pages.dev |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Select →button that derives that single session and jumps straight to Preview. Merging traces isn't well supported today, so the multi-select had no real payoff — and requiring users to scroll past a page of conversations to reach a footer button was the immediate annoyance.selected: Record<string, Record<string, true>>and theClaudeToggleSession/PiToggleSessionmessages are gone;ClaudeDerive/PiDerivenow carry{ path, sid }inline..preview-body--splitusedgrid-template-columns: 380px 1fr. Grid1frdefaults tominmax(auto, 1fr), so an unbreakable string in the transcript (long file path, pre block) would grow the right track past the viewport and put the whole preview page into horizontal scroll. Changed tominmax(0, 1fr)so the column is capped at the available width and.chat-view'soverflow-x: hiddencan clip properly. Pre-existing on main — surfaced while reviewing the new flow.Notes for reviewer
18pxmarker track); fixed by keeping the empty leadingrow-card__markerspan so the title + button fall in the1fr/autocells.derive_claude/derive_pistill take{ projectPath, sessionIds, includeThinking }; we just always send a one-elementsessionIds.Test plan
cargo tauri dev, open Claude tab on Home, clickSelect →on a session → preview opens with that one session.svelte-checkclean;cargo check -p toolpath-desktopclean.